broadway: Cast g_object_ref() to the right type

In GLib 2.56, g_object_ref() will check that you're assigning the return
value to a variable of the same type you're passing in.

x11: Balance an error trap

We are pushing an error trap, and never popping it from the stack.

Fix overzalous method annotations for drag and drop

Functional revert of commit 9c4892f2915.

Fixes introspection scanner warnings like:

  Warning: Gtk: gtk_drag_finish: Methods must belong to the same
  namespace as the class they belong to

That is, the gtk_drag_* functions cannot be methods as they have a
"GdkDragContext" as the instance parameter, and that is not a valid
type for the Gtk namespace.

This is not an introspected ABI change, as the generated introspection
data ignores the annotation.

Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>
https://bugzilla.gnome.org/show_bug.cgi?id=692152

Update Swedish translation

Update Polish translation

Add AGPL3-only licence to GtkAboutDialog

https://bugzilla.gnome.org/show_bug.cgi?id=792793

Update Friulian translation

Update Friulian translation

Update Galician translation

Update Galician translation

Update Chinese (Taiwan) translation

Update Chinese (Taiwan) translation

Merge branch 'wip/carlosg/fix-issue-1' into 'gtk-3-22'

wayland: Improve EOF detection when reading selections

See merge request GNOME/gtk!1

wayland: Improve EOF detection when reading selections

g_input_stream_read_bytes() roughly provides the same guarantees
than g_input_stream_read() wrt the number of bytes being possibly
read (i.e. it being a best effort, but no real guarantees).

Instead, rely on the 0-len read that we'd get at the end of the
transfer.

Fixes clipboard/DnD transfers possibly being cut short, resulting
on "Broken pipe" errors on the other side.

https://gitlab.gnome.org/GNOME/gtk/issues/1

Closes: #1

cssprovider: Only free bytes when they exist

css: Add a workaround for gtk_widget_override_font()

The problem here is that the CSS machinery expects font sizes to be in
pixels, but gtk_widget_override_font() provides a value in point and the
CSS machinery has no ability to query the DPI and convert.

This patch changes the dconversion DPI we use from a hardcoded 96 to the
default screen's DPI, which should work better than before.
This will of course not listen to changes in the default screen's DPI,
but that shouldn't be a problem.

People who want to workaround this should use gtk_widget_override_font()
with a font that has an absolute size set via
pango_font_description_set_absolute_size (size * PANGO_SCALE *
                                          gdk_screen_get_resolution (screen));

https://bugzilla.gnome.org/show_bug.cgi?id=774248

Update Finnish translation

Update Catalan translation

Update Catalan translation

iconview: Fix a problem with the previous change

After commit ffef28a7e8d5ffef7de6a3baccb30b0021b6b0ff,
gtk-icon-browser was spewing critical warnings when
changing sections. Avoid that by respecting the return
value of gtk_tree_model_get_iter.

wayland: Add support for BTN_STYLUS3

BTN_STYLUS3 is defined by the Linux 4.15 kernel and is sent when the
third button on a stylus is pressed. At the moment, only Wacom's "Pro
Pen 3D" has three stylus buttons. Pressing this button triggers a button
8 event to be sent under X11, so we use the same mapping here.

https://bugzilla.gnome.org/show_bug.cgi?id=790033

tests: Don't add testswitch twice in Makefile.am

menu: Guard against NULL toplevel

This can happen, as indicated by GtkMenu explicitly connecting to
::destroy of its toplevel window. Do the same thing in GtkComboBox.

Update Afrikaans translation

Update Afrikaans translation

Widget: Don’t call reset() on NULL EventController

GtkGesture is a GtkEventController. gtk_event_controller_dispose() calls
_gtk_widget_remove_controller(). That NULLs the pointer-to-Controller in
our EventControllerData but does not delete said ECData from our GList.

Subsequently, if that same Widget gets unparent()ed, that method calls
unset_state_flags(), which leads to doing reset_controllers() if we are
insensitive. Now, unlike most most other loops over the GList of ECData,
reset_controllers() does not skip nodes whose pointer-to-Controller is
NULL. So, we call gtk_event_controller_reset(NULL) and get a CRITICAL.

This surfaced in a gtkmm program. The Gesture is destroyed before the
Widget. The Widget then gets dispose()d, which calls unparent()… boom.
I didn’t find an MCVE yet but would hope this logic is correct anyway:

The simplest fix is to make the loop in gtk_widget_reset_controllers()
skip GList nodes with a NULL Controller pointer, like most other such
loops, so we avoid passing the NULL to gtk_event_controller_reset().

In other, live cases, _gtk_widget_run_controllers() loops over the GList
and removes/frees nodes having NULL Controllers, so that should suffice.
But this clearly was not getting a chance to happen in the failing case.

https://bugzilla.gnome.org/show_bug.cgi?id=792624

Update Esperanto translation

GtkGestureMultiPress: check event state before emiting released signal

Fix bug 771986 "Inconsistent 'row-activated' signal emission before \
drag'n'drop, 'activate-on-single-click'=TRUE, 'reorderable'=TRUE"

Update Hungarian translation

Update Hungarian translation

iconview: Fix updates with filter models

Filter models rely on views taking a ref on every node
they care about. GtkIconView was not doing that. Amazingly,
this has never shown up in a bug so far, until I spotted
the fallout in gnome-font-viewer.

Add a test for ::row-changed vs node refs

Test that filter models propagate ::row-changed if there is
an external reference on the node, and not otherwise. This
is showing up in buggy icon view behaviour, where the icon
view is not redrawing if the content changes in a model that
is below a filter model.

font chooser: Fix property notification

We must notify the font and font-desc properties when the
list selection changes, and return NULL values for them
when there is no selection in the list.

font chooser dialog: Improve sensitivity handling

Enable the select button when a font is selected, as it should be.

font chooser: Small revision of the UI

Show only the font names in the list, in their own
font, in order to make the list less noisy.

gtk/queryimmodules.c: Make the output deterministic.

Whilst working on the Reproducible Builds effort [0], we noticed that
queryimmodules generates non-reproducible output as it iterates over the
filesystem without sorting.

Patch attached.

 [0] https://reproducible-builds.org/

Signed-off-by: Chris Lamb <lamby@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786528

tree model sort: Fix set_sort_column

We were failing to change the sort order for the
default sort column in some cases. Fix that, and
add a testcase for this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=792459

Add a testcase for the previous fix

tree model sort: Fix initial default sort func

gtk_tree_sortable_has_default_sort_func should return
FALSE initially.

Update Polish translation

emoji: Skip overly wide fallback rendering

Some emoji fonts (such as Emoji One), render Emoji sequences
such as some of the family variations using multiple individual
glyphs. This rendering is too wide and breaks our grid layout.
Therefore, we will just skip any sequence whose rendering is
more than twice as wide as a simple smiley.

GdkMonitor: Fix link to nonexistent Display method

There is no gdk_display_get_monitors(). Instead, we have to use
gdk_display_get_n_monitors() and gdk_display_get_monitor(int).

HeaderBar: Explain use in conjunction w/ GtkWindow

After hinting how good GtkHeaderBar is for GtkWindow, let’s link to
the latter and indicate how users can make the two work together.

Window: Mention GtkHeaderBar in set_titlebar() doc

This is the typical thing passed here and what most users want, so we
should mention it here, rather than requiring users to figure it out.

Window: Clarify resize() doc about titlebar widget

Clarify the reference to HeaderBar, as it applies to any custom title
widget; HeaderBar is only the most common one used. Also, fix a typo.

wayland: Use portable sed syntax in gdk/wayland/Makefile.am

Replace non-portable \<, \>, \+ with \(, \), \{, \} to avoid build
failure on systems not using GNU sed.

https://bugzilla.gnome.org/show_bug.cgi?id=792279

GtkCssImageSurface: add cache for the last drawn size.

Keep a copy of the scaled image to speed up rendering multiple times
the image at the same size.

Adwaita: Fix typo lenght => length in SASS

Update Icelandic translation

Update Icelandic translation

ScrolledWindow: add() before remove() in snippet

We can't remove() a child widget that wasn't already add()ed, of course.

file button: Don't leak rows

The file chooser button manually manages the memory of
data in its model, so it needs to explicitly free the
rows.

Remove a dead macro

GCs have been dead for a long time.

Make a few more code samples compile

entry: Remove some questionable code examples

widget: Make all code snippets compile

liststore: Make a code snippet compile

searchbar: Make a code snippet compile

menuitem: Make a code snippet compile

expander: Make a code snippet compile

treeviewcolumn: Make a code snippet compile

notebook: Make a code snippet compile

scrolledwindow: Fix code sample compilation

Drop an unused enum value

We haven't had a GtkWindow::frame-event since the
linux-fb backends demise.

filechooser: Add missing array annotations to add_choice()

iconview: Prevent ownership transfer of cell out-param in get_item_at_pos()

Add test for GtkActionable GtkListBoxRow.

https://bugzilla.gnome.org/show_bug.cgi?id=741633

Make GtkListBoxRow GtkActionable.

https://bugzilla.gnome.org/show_bug.cgi?id=741633

configure: Don't declare functions in AC_TRY_COMPILE and AC_LANG_PROGRAM

Both AC_TRY_COMPILE and AC_LANG_PROGRAM put code passed to their second
arguments to the body of the main function. This means that we cannot
and should not declare functions there, or we end up checking whether
the compiler support nested functions instead of whether a compiler or
linker flag is supported.

GCC supports nested functions and tests succeed. Clang doesn't support
nested functions, so tests fail and -fvisibility=hidden won't be used.
This means that functions which are not intended to be used by other
programs, such as gtk_menu_tracker*, gtk_action_observ*,
gtk_menu_muxer_*, become global symbols with default visibility.

GNOME Shell has a private library libgnome-shell-menu.so, which also has
symbols gtk_menu_tracker*, gtk_action_observ*, gtk_menu_muxer_* that are
intended to be used by GNOME Shell itself. When GNOME Shell still used
Autotools build system, the executable gnome-shell explicitly linked to
libgnome-shell-menu.so, so the linker loaded libgnome-shell-menu.so
before libgtk-3.so.0 and GNOME Shell used correct symbols from its
private library.

However, after GNOME Shell switched to Meson build system, gnome-shell
executable no longer lists libgnome-shell-menu.so as its dependency.
Even if we adds it to the build file, it won't be listed in DT_NEEDED of
gnome-shell because Meson uses -Wl,--as-needed by default. This causes
the runtime linker to load libgtk-3.so.0 before libgnome-shell-menu.so
and symbols gtk_menu_tracker*, gtk_action_observ*, gtk_menu_muxer_* are
bound to libgtk-3.so.0 instead of libgnome-shell-menu.so. GNOME Shell
hangs when opening more than one window because it uses functions from
the wrong library.

This problem is already fixed in OpenBSD ports. The article describing
it can be found on OpenBSD Journal with this link:
https://undeadly.org/cgi?action=article;sid=20170930133438

https://bugzilla.gnome.org/show_bug.cgi?id=791943

gtk: remove non-breaking space, probably there by mistake

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: fix some formatting inconsistencies along the way

Mostly spaces.

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: remove useless <para>

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: swap border-width and border-left shorthands, to keep going clockwise

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: Fix refentry and refname

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: Ensure the first column does not break for every table

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: Get rid of non-breaking hyphens

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: Get rid of non-breaking spaces

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

css docs: Fix some angle brackets for percentage

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

window: Avoid redundantly getting the GDK window

meta-pick of commit c1573a1fda7c3b917b6809795b9668629fe58883: the
variable gdk_window is used in a check, but we may as well use it again
to avoid calling _gtk_widget_get_window() again unnecessarily.

themes: Regenerate CSS to reflect recent commits

HighContrast: Avoid expander↔combobox interference

as per commit b4b9aa1e5fab6a1f8c27b768813ab4ed04f3cd6f

css docs: Fix duplicate border‑right‑width

Signed-off-by: Arnaud Rebillout <elboulangero@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=791710

Adwaita: Avoid expander<>combobox interference

Putting a combobox in an expander was causing the combo arrow
to go sideways. Increase the specificity with which we address
the expander arrow to avoid that.

Slovenian update ...

Slovenian update ...

fontchooser: Add global keynav

Starting to type should focus the search entry and start
a search.

Add some deprecation annotations

These functions have been removed in master, and their
replacements are already available here.

font button: Some followup cleanups

Don't refer to the font-name property in docs either, and
don't use deprecated API internally.

font button: Deprecate the font-name property

It has long been redundant with the GtkFontChooser::font
property.

tests: Stop using gtk_font_button_get_font_name

We can instead use the GtkFontChooser::font property.

Range: Fix inverted vert scrolling on vert Ranges…

The change in the previous patch should only be applied when the Range
is oriented horizontally.

https://bugzilla.gnome.org/show_bug.cgi?id=737175

Range: Fix inverted vert scrolling on horiz Ranges

Users expect, & previous patches have tried to assure, that scrolling up
over a horizontal Range will cause the value to increase & vice-versa.
But the path using directions was still negating the delta & decreasing
the value on scrolling up. This could be seen on Win32 or X without XI2.

So, only negate the delta when scrolling down (or left), not up, so that
scrolling up (or right) will make the value increase for any event type.

https://bugzilla.gnome.org/show_bug.cgi?id=737175#c5

font chooser: Stop listening to style-updated

The only time a style-updated indicates we need
to reload fonts is when it is synthesized by GtkSettings
in response to a fontconfig timestamp change, but
we are listening to those already, anyway.

Avoid calling unsetenv too late

Stash the DESKTOP_AUTOSTART_ID env var in a constructor,
before any threads have been created.

https://bugzilla.gnome.org/show_bug.cgi?id=790963

wayland: Handle scale 3 for cursors

This is just a bandaid solution to make scale 3 work.
If people seriously want to go for scales larger than
that, we need a better solution.

inspector: Allow to bump scale to 3

Might as well allow this, for 'large scale testing'.

cups: Warn only once if colord is not available

No need to keep complaining about this.

https://bugzilla.gnome.org/show_bug.cgi?id=791650

Revert "css: Replace _AFFECTS_TEXT with _TEXT_SIZE and _TEXT_CLIP"

This reverts commit 492469a94af35c2b102bcf7f07046ef2d09b2b3d.

People have been reporting missing invalidations with this change.

Revert "css: Fix segfault when GtkCssStyleChange is NULL on gtk_widget_real_style_updated()"

This reverts commit 7e253f4bd3977a67cf7040ce11e24b9d06100f6a.

Don't place dnd cursor in placeholder text

That is just misleading - the entire placeholder text is going
to be replaced by the drop.

listbox: Fix a crash during keynav

The code was asserting something that was not always holding
true. We can hit row == NULL here on page-up too. Handle that
case by moving to the first row.

https://bugzilla.gnome.org/show_bug.cgi?id=791549